-
Notifications
You must be signed in to change notification settings - Fork 2.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Expose getFileTypeIconAsURL helper for react-file-type-icons #27940
base: master
Are you sure you want to change the base?
Conversation
📊 Bundle size report🤖 This report was generated against c0214bcb7d9369db75eb842adf4253893bc0e160 |
cc: @behowell for visibility |
Asset size changesUnable to find bundle size details for Baseline commit: ab90189 Possible causes
Recommendations
|
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. Latest deployment of this branch, based on commit acf0be6:
|
@micahgodbolt is this something to help with while Ben is oof? cc: @grace-zou |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm. we should have a convo about how to support file-type icons within v9 for the future.
/azp run |
Azure Pipelines successfully started running 4 pipeline(s). |
I have approved and merged this PR: #27336. Can you pull from master and update this PR? |
Previous Behavior
File Type Icons can't be used with Fluent 9 unless you import Fluent 8
<Icon />
or use React to dangerously set inner HTML.New Behavior
Expose a new helper method so Fluent 9 code can use
<img src={getFileTypeIconAsURL()} />
.Related Issue(s)